-
Notifications
You must be signed in to change notification settings - Fork 4.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[RNMobile] Improve unsupported block message for reusable block #32618
[RNMobile] Improve unsupported block message for reusable block #32618
Conversation
The unsupported block message has been updated and the convert to regular blocks button has been also included.
@iamthomasbishop I'd appreciate your input on this change, let me know if you have any suggestions, thanks 🙇 ! |
Size Change: +2.57 kB (0%) Total Size: 1.04 MB
ℹ️ View Unchanged
|
@fluiddot Looks pretty good. I think we could make the "alternatively" part a bit clearer, maybe something along the lines of:
What do you think? |
Sure! I'll update that part with your suggestion, thanks! |
I updated the screenshots and video capture of the PR description with the last changes mentioned in this comment. @iamthomasbishop I finally changed the message to |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great! I tested on both platforms.
The only issue I ran into was #32699, which is a known issue.
* Improve unsupported block message for reusable block The unsupported block message has been updated and the convert to regular blocks button has been also included. * Update unsupported block message of reusable block
* Release script: Update react-native-editor version to 1.55.0 * Release script: Update with changes from 'npm run core preios' * Release script: Update react-native-editor version to 1.55.1 * Release script: Update with changes from 'npm run core preios' * Fix: RNMobile borderRadius value setting (#32717) * [RNMobile] Improve unsupported block message for reusable block (#32618) * Improve unsupported block message for reusable block The unsupported block message has been updated and the convert to regular blocks button has been also included. * Update unsupported block message of reusable block * Update react-native-editor changelog Co-authored-by: Antonis Lilis <antonis.lilis@automattic.com> Co-authored-by: Enej Bajgoric <enej.bajgoric@automattic.com>
* Release script: Update react-native-editor version to 1.55.0 * Release script: Update with changes from 'npm run core preios' * Release script: Update react-native-editor version to 1.55.1 * Release script: Update with changes from 'npm run core preios' * Fix: RNMobile borderRadius value setting (#32717) * [RNMobile] Improve unsupported block message for reusable block (#32618) * Improve unsupported block message for reusable block The unsupported block message has been updated and the convert to regular blocks button has been also included. * Update unsupported block message of reusable block * Update react-native-editor changelog * Release script: Update react-native-editor version to 1.55.2 * Release script: Update with changes from 'npm run core preios' * Fix incorrect block insertion point after blurring post title (#32831) Selecting the title selection status within the same `useSelect` for reusable blocks caused the title selection status to become stale, due to the dependency array for the `useSelect` hook. The staleness caused the block insertion point to show up in the incorrect location after blurring the title text input. * Update changelog * Properly handle 404 errors while publishing Android artifacts (#32860) Co-authored-by: Antonis Lilis <antonis.lilis@automattic.com> Co-authored-by: Carlos Garcia <fluiddot@gmail.com> Co-authored-by: Enej Bajgoric <enej.bajgoric@automattic.com> Co-authored-by: Oguz Kocer <oguzkocer@users.noreply.github.com>
gutenberg-mobile
PR: wordpress-mobile/gutenberg-mobile#3621Description
The unsupported block message has been updated and the convert to regular blocks button has been also included.
How has this been tested?
Screenshots
reusable-block-convert-bottom-sheet.mp4
Types of changes
Enhancement
Checklist:
*.native.js
files for terms that need renaming or removal).